+++ This bug was initially created as a clone of Bug #1622660 +++

Filling as a good first bug to learn workflows.

return Transition::ToUnbuffered(ICOState::FINISHED_RESOURCE,
                                    ICOState::READ_RESOURCE, toRead);
  } else {
    // Make sure we have a sane size for the bitmap information header.
    int32_t bihSize = LittleEndian::readUint32(aData);```
do not use 'else' after 'return'

https://searchfox.org/mozilla-central/source/image/decoders/nsICODecoder.cpp#359-364

As the change is trivial, it is just to learn how to contribute to Firefox.

Found by http://clang.llvm.org/extra/clang-tidy/checks/readability-else-after-return.html

Tutorial to contribute:
https://firefox-source-docs.mozilla.org/tools/docs/contribute/how_to_contribute_firefox.html

Please don't ask for the bug to be assigned. It will be automatically assigned to the first patch.

https://hg.mozilla.org/mozilla-central/rev/e3ba3bcc2dae